Skip to content

GLSP-1586: Adjust to new MenuItem API#266

Merged
tortmayr merged 2 commits into
masterfrom
glsp-1586
Oct 10, 2025
Merged

GLSP-1586: Adjust to new MenuItem API#266
tortmayr merged 2 commits into
masterfrom
glsp-1586

Conversation

@tortmayr

@tortmayr tortmayr commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

What it does

Adjust Theia GLSP context menu service for new MenuItem API. Ensure that isEnabled,isVisible and isToggled are properly forwarded to the Theia Menu item

Part of eclipse-glsp/glsp#1586

Requires: eclipse-glsp/glsp-client#450
Requires: eclipse-glsp/glsp-server-node#116

How to test

  • Context menu should now have a new "Readonly mode" entry that allows toggle of readonly
  • The server-side provided context menu has a third item in the new section that has been hidden on server side
    -> make sure that it is not visible

Follow-ups

Changelog

  • This PR should be mentioned in the changelog
  • This PR introduces a breaking change (if yes, provide more details below for the changelog and the migration guide)

Adjust Theia GLSP context menu service for new MenuItem API.
Ensure that isEnabled,isVisible and isToggled are properly forwarded to
the Theia Menu item

Part of eclipse-glsp/glsp#1586

Requires: eclipse-glsp/glsp-client#450
Requires: eclipse-glsp/glsp-server-node#116

@martin-fleck-at martin-fleck-at left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tortmayr tortmayr merged commit 5fdf3f8 into master Oct 10, 2025
4 checks passed
@tortmayr tortmayr deleted the glsp-1586 branch October 10, 2025 09:08
@github-actions github-actions Bot mentioned this pull request Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants